Skip to content

Expand regex to match n param decrypt function#696

Merged
Stypox merged 6 commits intoTeamNewPipe:devfrom
XiangRongLin:decrypt_pattern
Aug 1, 2021
Merged

Expand regex to match n param decrypt function#696
Stypox merged 6 commits intoTeamNewPipe:devfrom
XiangRongLin:decrypt_pattern

Conversation

@XiangRongLin
Copy link
Copy Markdown
Collaborator

  • I carefully read the contribution guidelines and agree to them.
  • I have tested the API against NewPipe.
  • I agree to create a pull request for NewPipe as soon as possible to make it compatible with the changed API.

Temporary solution by expanding the regex, which extracts the decrypt function.

Depending on how long/difficult a proper solutions turns out, this could be unneeded. @Stypox

@XiangRongLin XiangRongLin added bug Issue or PR is related to a bug YouTube Service, https://www.youtube.com/ labels Jul 29, 2021
@XiangRongLin
Copy link
Copy Markdown
Collaborator Author

@Stypox I took the liberty to add your solution here directly

Copy link
Copy Markdown
Member

@Stypox Stypox left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, looks good to me

Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/utils/Parser.java Outdated
Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/utils/StringUtil.java Outdated
Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/utils/StringUtil.java Outdated
@AudricV AudricV dismissed their stale review July 29, 2021 19:35

Outdated review

Comment thread extractor/src/main/java/org/schabi/newpipe/extractor/utils/StringUtils.java Outdated
@Stypox Stypox merged commit c97a19d into TeamNewPipe:dev Aug 1, 2021
@XiangRongLin XiangRongLin deleted the decrypt_pattern branch August 1, 2021 08:49
@Thewisem
Copy link
Copy Markdown

Thewisem commented Aug 1, 2021

Hmmm how can I get the .apk

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Issue or PR is related to a bug YouTube Service, https://www.youtube.com/

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants